-
Couldn't load subscription status.
- Fork 567
Don’t leave trailing whitespace section #7
Conversation
Brings in the no trailing whitespace section from github/objective-c-style-guide#58.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/nope
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I normally try to be understanding of your multiline spacing tendencies, but this is just input into a rendered document. The blank line doesn't do anything, and the header syntax (####) separates things well enough on its own.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I was wondering about this too, @robrix…
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Newline + #### makes it much easier for me to spot and parse the sections than newline alone.
I accept that this may not be a problem for you. Does this formatting introduce issues (technical or otherwise) for you?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Does this formatting introduce issues (technical or otherwise) for you?
No, I'm personally just curious.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That was primarily directed at @jspahrsummers 😊
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
|
|
Don’t leave trailing whitespace section

Brings in the no trailing whitespace section from github/objective-c-style-guide#58.